Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add Generic ConfigDB Validation via YANG Design Doc #1072

Merged
merged 7 commits into from
Oct 12, 2022
Merged

Add Generic ConfigDB Validation via YANG Design Doc #1072

merged 7 commits into from
Oct 12, 2022
+221 −0

Conversation

isabelmsft
Copy link
Contributor

@isabelmsft isabelmsft commented Aug 25, 2022 •
edited
Loading

This PR contains the ConfigDB Validation via YANG HLD.

PR title (included in 202211 branch) state context
[sonic-utilities] Added CLI to enable/disable yang config validation in ConfigDB GitHub issue/pull request detail GitHub pull request check contexts
[sonic-buildimage] Add yang_config_validation to DEVICE_METADATA yang model GitHub issue/pull request detail GitHub pull request check contexts
[sonic-buildimage] Add yang_config_validation to minigraph.py GitHub issue/pull request detail GitHub pull request check contexts
[sonic-utilities] Added support for ConfigDB updates via YANG validation - including validation for portchannel, loopback, and vlan interfaces GitHub issue/pull request detail GitHub pull request check contexts
[sonic-utilities] Add YANG validation for portchannel member and mgmt interface GitHub issue/pull request detail GitHub pull request check contexts
[sonic-utilities] YANG validation for ConfigDB Updates: MIRROR_SESSION use case GitHub issue/pull request detail GitHub pull request check contexts
[sonic-utilities] YANG Validation for ConfigDB Updates: TACPLUS, TACPLUS_SERVER, AAA, VLAN_SUB_INTERFACE, NETWORKING_METADATA tables + decorated validated_mod_entry GitHub issue/pull request detail GitHub pull request check contexts
[sonic-utilities] YANG Validation for ConfigDB Updates: DEVICE_METADATA, SNMP, SNMP_COMMUNITY GitHub issue/pull request detail GitHub pull request check contexts
[sonic-utilities] YANG Validation for ConfigDB Updates: PORT_STORM_CONTROL, PORT_QOS_MAP, BUFFER_PROFILE, BUFFER_PG, BUFFER_QUEUE, BUFFER_POOL, FEATURE, DEFAULT_LOSSLESS_BUFFER_PARAMETER tables GitHub issue/pull request detail GitHub pull request check contexts
PR title (to be cherrypicked into 202211 branch) state context
[sonic-utilities] YANG Validation for ConfigDB Updates: WARM_RESTART, SFLOW_SESSION, SFLOW, VXLAN_TUNNEL, VXLAN_EVPN_NVO, VXLAN_TUNNEL_MAP, MGMT_VRF_CONFIG, CABLE_LENGTH, VRF tables GitHub issue/pull request detail GitHub pull request check contexts
[[sonic-utilities] YANG Validation for ConfigDB Updates: RADIUS_SERVER tables ](sonic-net/sonic-utilities#2604) GitHub issue/pull request detail GitHub pull request check contexts
[sonic-utilities] YANG Validation for ConfigDB Updates: CONSOLE_SWITCH, CONSOLE_PORT, KUBERNETES_MASTER, BREAKOUT_CFG, NTP_SERVER, SFLOW_COLLECTOR, MCLAG_DOMAIN, MCLAG_INTERFACE, MCLAG_UNIQUE_IP, MUX_CABLE, STATIC_NAT, STATIC_NAPT, NAT_POOL, NAT_BINDINGS, NAT_GLOBAL tables GitHub issue/pull request detail GitHub pull request check contexts
YANG Model GitHub Issue state
[yang] MIRROR_SESSION GitHub issue/pull request detail
[yang] VLAN_SUB_INTERFACE GitHub issue/pull request detail
[yang] RADIUS_SERVER, RADIUS missing YANG models GitHub issue/pull request detail
[yang] SNMP_AGENT_ADDRESS_CONFIG, SNMP_USER missing YANG models GitHub issue/pull request detail
[yang] SNMP GitHub issue/pull request detail
[yang] CHASSIS_MODULE GitHub issue/pull request detail
[yang] XCVRD_LOG GitHub issue/pull request detail

@qiluo-msft
Copy link
Contributor

@isabelmsft Please add all the related PRs table into HLD PR description. So in future, audience will have a great overview of the project status, and easily cherry-pick them to any other branches.

For example: #1024

@isabelmsft
Copy link
Contributor Author

@isabelmsft Please add all the related PRs table into HLD PR description. So in future, audience will have a great overview of the project status, and easily cherry-pick them to any other branches.

For example: #1024

I have added the table for related PRs to this PR description

qiluo-msft
qiluo-msft previously approved these changes Oct 5, 2022
View reviewed changes
@qiluo-msft qiluo-msft merged commit a181a5a into sonic-net:master Oct 12, 2022
@oleksandrivantsiv oleksandrivantsiv mentioned this pull request Jun 8, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@qiluo-msft qiluo-msft qiluo-msft approved these changes

@dgsudharsan dgsudharsan dgsudharsan approved these changes

Assignees
No one assigned
Labels
Request for 202211 Branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@isabelmsft @qiluo-msft @dgsudharsan @zhangyanzhao